-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use IMU measurements to set submodel base state in RDE #793
Conversation
CHANGELOG.md.orig
Outdated
@@ -0,0 +1,522 @@ | |||
# Changelog |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove this file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
CHANGELOG.md
Outdated
@@ -15,6 +15,7 @@ All notable changes to this project are documented in this file. | |||
- Add the support of QP problems with no constraint in `QPInverseKinematics` and `QPTSID` (https://github.com/ami-iit/bipedal-locomotion-framework/pull/784) | |||
- Implement `blf-joints-grid-position-tracking` application in `utilities` (https://github.com/ami-iit/bipedal-locomotion-framework/pull/787) | |||
- Add the possibility to resample the contact in a given contact list (https://github.com/ami-iit/bipedal-locomotion-framework/pull/788) | |||
- Set submodel states from IMUs in RDE (Use IMU measurements to set submodel base state in RDE) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you rebase the PR on top of master and move this line to unreleased?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
bc97540
to
b8163a9
Compare
@GiulioRomualdi after the rebase the CI fails. |
@traversaro I requested also your review to be sure about the implemented formulas. If you can check the code here
|
Th computation seems correct. However, formula in
|
b8163a9
to
45d45ba
Compare
Hi @isorrentino can you rebase the PR on top of master? |
…urement dynamics for the same sensor
a665e51
to
0a44679
Compare
Done. |
This PR implements what is described in #70 and #95.
In addition, the PR introduces a code restructure to have the possibility to define more than one measurement dynamics for a given sensor.
The tests have been modified accordingly.
[UPDATE] I added also the possibility of having the friction torques as a measurement for the estimator.